domain/search/src/main/java/com/teamsolply/solply/search/MyClass.kt (1)

3-3: 의미 없는 플레이스홀더 클래스 제거 검토

MyClass가 비어 있는 상태로 남아 있으면 추후 실제 구현과 충돌하거나 불필요한 참조가 생길 수 있습니다. 지금 단계에서 사용 목적이 없다면 파일을 제거하거나, 필요한 실제 도메인 로직/모델로 대체하는 편이 좋겠습니다.

feature/serach/src/main/java/com/teamsolply/solply/search/component/SearchItem.kt (1)

97-104: 프리뷰용 PlaceType 값 오타

실제 enum 값이 WALKING인데 "WALK"으로 찾고 있어 항상 PlaceType.ALL로 fallback 됩니다. 프리뷰라도 의도와 다르니 값 교정이 좋겠습니다.

-            val walk = PlaceType.entries.firstOrNull { it.name == "WALK" } ?: PlaceType.ALL
+            val walk = PlaceType.entries.firstOrNull { it.name == "WALKING" } ?: PlaceType.ALL

feature/main/build.gradle.kts (1)

14-14: 모듈 이름 철자 serach → search로 정비 필요

새 기능이 “Search”를 의미하는데 모듈/의존성 이름이 모두 serach로 등록되어 있습니다. 지금 바로 정리하지 않으면 이후 레이어 전반에 오타가 굳어져 유지보수가 어려워집니다. 모듈 디렉터리와 Gradle 식별자를 search로 통일해 주세요.

feature/serach/src/main/java/com/teamsolply/solply/search/SearchContract.kt (2)

13-15: Public API에서 PersistentList 노출 지양(ImmutableList로 완화 권장)

구현 세부(퍼시스턴트 컬렉션) 노출을 줄이기 위해 PersistentList 대신 ImmutableList(또는 단순 List)로 표면화하는 것이 좋습니다. 기본값은 그대로 persistentListOf()를 사용해도 됩니다.

 import kotlinx.collections.immutable.PersistentList
+import kotlinx.collections.immutable.ImmutableList
 import kotlinx.collections.immutable.persistentListOf

 data class SearchState(
     val query: String = "",
     val isLoading: Boolean = false,
-    val results: PersistentList<SearchItemUi> = persistentListOf(),
+    val results: ImmutableList<SearchItemUi> = persistentListOf(),
     val selectedTownId: Long? = null
 )

Also applies to: 7-8

---

21-26: id 네이밍을 placeId로 명확화(가독성/일관성 향상)

SearchItemUi.id와 ClickItem(id)는 의미상 장소 식별자이므로 placeId로 변경 시 의도가 선명해집니다. 초기 단계라면 지금 반영하는 편이 장기 유지보수에 유리합니다(호출부 전반 수정 필요).

 data class SearchItemUi(
-    val id: Long,
+    val placeId: Long,
     val name: String,
     val tag: PlaceType,
     val address: String,
     val imageUrl: String
 )

 sealed interface SearchIntent : UiIntent {
     data class QueryChanged(val value: String) : SearchIntent
     data object ClearQuery : SearchIntent
     data object Submit : SearchIntent
-    data class ClickItem(val id: Long) : SearchIntent
+    data class ClickItem(val placeId: Long) : SearchIntent
     data object ClickNoResult : SearchIntent
     data object Retry : SearchIntent
 }

Also applies to: 32-33

feature/serach/src/main/java/com/teamsolply/solply/search/SearchScreen.kt (4)

41-50: LaunchedEffect 중복 및 키 보강(뷰모델에 바인딩하고 하나로 합치기)

동일 키(Unit)로 두 개의 LaunchedEffect를 사용 중입니다. viewModel을 키로 하나의 블록에 합치면 수명 관리가 명확해지고 재수집 이슈를 줄일 수 있습니다.

-    LaunchedEffect(Unit) {
-        viewModel.sideEffect.collectLatest { se ->
-            when (se) {
-                is SearchSideEffect.NavigateToPlaceDetail -> navigateToPlaceDetail(se.townId, se.placeId)
-                SearchSideEffect.NavigateToNoResult -> onNoPlaceClick()
-            }
-        }
-    }
-    LaunchedEffect(Unit) { viewModel.sendIntent(SearchIntent.Retry) }
+    LaunchedEffect(viewModel) {
+        viewModel.sendIntent(SearchIntent.Retry)
+        viewModel.sideEffect.collectLatest { se ->
+            when (se) {
+                is SearchSideEffect.NavigateToPlaceDetail -> navigateToPlaceDetail(se.townId, se.placeId)
+                SearchSideEffect.NavigateToNoResult -> onNoPlaceClick()
+            }
+        }
+    }

---

121-130: 불필요한 toImmutableList 제거 및 리스트 key 추가

이미 PersistentList이므로 복사/변환 불필요합니다. 또한 itemsIndexed에 key를 제공해 항목 식별 안정성과 성능을 개선하세요.

-import kotlinx.collections.immutable.toImmutableList
@@
-                val items = state.results.toImmutableList()
-                itemsIndexed(items) { _, item ->
+                itemsIndexed(
+                    state.results,
+                    key = { _, item -> item.id } // placeId로 이름 변경 시 item.placeId
+                ) { _, item ->
                     SearchItem(
-                        placeName = item.name,
-                        placeTag = item.tag,
-                        placeAddress = item.address,
-                        placeImageUrl = item.imageUrl,
-                        onClick = { onClickItem(item.id) }
+                        placeName = item.name,
+                        placeTag = item.tag,
+                        placeAddress = item.address,
+                        placeImageUrl = item.imageUrl,
+                        onClick = { onClickItem(item.id) } // placeId로 이름 변경 시 item.placeId
                     )
                 }

Also applies to: 29-29

---

82-84: 하드코딩 문자열 → stringResource로 i18n/접근성 강화

문자열을 리소스로 분리하고 stringResource를 사용하세요. 접근성용 contentDescription도 리소스화 권장합니다.

 import androidx.compose.ui.res.painterResource
+import androidx.compose.ui.res.stringResource
+import com.teamsolply.solply.search.R
@@
-        SolplyTopBar(
-            barText = "검색하기",
+        SolplyTopBar(
+            barText = stringResource(R.string.search_title),
             onBackButtonClick = onBack
         )
@@
-            Text(
-                text = "검색 결과 ${state.resultCount}개",
+            Text(
+                text = stringResource(R.string.search_result_count, state.resultCount),
                 style = SolplyTheme.typography.button14M,
                 color = SolplyTheme.colors.gray800,
                 modifier = Modifier.padding(start = 20.dp, top = 32.dp, bottom = 16.dp)
             )
@@
-                        Text(
-                            text = "찾는 장소가 없어요",
+                        Text(
+                            text = stringResource(R.string.search_no_result),
                             style = SolplyTheme.typography.button14M,
                             color = SolplyTheme.colors.gray700,
                             modifier = Modifier.weight(1f)
                         )
                         Icon(
                             painter = painterResource(id = com.teamsolply.solply.designsystem.R.drawable.ic_arrow_right_icon),
-                            contentDescription = "arrow-right-icon",
+                            contentDescription = stringResource(R.string.cd_arrow_right),
                             tint = SolplyTheme.colors.gray700
                         )

추가 리소스(참고):

<!-- feature/serach/src/main/res/values/strings.xml -->
<resources>
    <string name="search_title">검색하기</string>
    <string name="search_result_count">검색 결과 %1$d개</string>
    <string name="search_no_result">찾는 장소가 없어요</string>
    <string name="cd_arrow_right">다음으로 이동</string>
</resources>

Also applies to: 101-106, 147-158, 21-22

---

1-1: 모듈 경로 오타: feature/serach → feature/search로 정리 권장

경로 오타는 IDE 검색/탐색, 스크립트, 신규 팀원의 탐색성을 해칩니다. 모듈/패키지/gradle 설정 일괄 리네임을 고려해주세요.

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro